Skip to content

feat: make use of IPreloadableNotifier - #2452

Merged
nickvergessen merged 1 commit into
masterfrom
feat/preload-many-notifications
Sep 3, 2026
Merged

feat: make use of IPreloadableNotifier#2452
nickvergessen merged 1 commit into
masterfrom
feat/preload-many-notifications

Conversation

@st3iny

@st3iny st3iny commented Aug 4, 2025

Copy link
Copy Markdown
Member

Comment thread lib/Push.php Outdated
Comment thread lib/Push.php Outdated
Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
@nickvergessen
nickvergessen force-pushed the feat/preload-many-notifications branch from a1e4c98 to a96dd07 Compare September 3, 2026 14:12
@nickvergessen
nickvergessen marked this pull request as ready for review September 3, 2026 14:12
@nickvergessen nickvergessen added this to the Nextcloud 36 milestone Sep 3, 2026
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

馃悽 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 9159 was 9041 (+1.3%)
Please check your code again. If you added a new test this can be expected and the base value in tests/Integration/base-query-count.txt can be increased.

@nickvergessen
nickvergessen merged commit c528034 into master Sep 3, 2026
49 checks passed
@nickvergessen
nickvergessen deleted the feat/preload-many-notifications branch September 3, 2026 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants